-
-
Notifications
You must be signed in to change notification settings - Fork 18.7k
TST: coverage for skipped tests in io/formats/test_to_html.py #22888
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Hello @simonjayhawkins! Thanks for submitting the PR.
|
|
changing: pandas/pandas/io/formats/html.py Line 235 in e45a6c1
to: sentinel = False would pass all tests. |
I have reduced the scope of the skipped tests to the truncation part and removed the there are other tests that check that the style is applied. |
can you rebase / tests are failng |
Codecov Report
@@ Coverage Diff @@
## master #22888 +/- ##
==========================================
+ Coverage 92.25% 92.27% +0.01%
==========================================
Files 161 161
Lines 51186 51186
==========================================
+ Hits 47222 47230 +8
+ Misses 3964 3956 -8
Continue to review full report at Codecov.
|
thanks! |
xref #11060 |
git diff upstream/master -u -- "*.py" | flake8 --diff
There are currently 3 skipped tests in
io/formats/test_to_html.py
;test_to_html_truncate
,test_to_html_truncate_multi_index
andtest_to_html_truncate_multi_index_sparse_off
Only
test_to_html_truncate
is required to provide regression coverage for #22786However, it is probably appropriate for any changes to
test_to_html_truncate
to also be applied to the other two skipped tests within the scope of this PR.cc @jreback